Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: fetch displayTitle and displaySubtitle for translations DHIS2-16216 #2929

Merged
merged 4 commits into from
Mar 6, 2024

Conversation

edoardo
Copy link
Member

@edoardo edoardo commented Dec 7, 2023

Fixes DHIS2-16216

Requires dhis2/analytics#1597


Key features

  1. Use translated title and subtitle in charts

Description

When translations are available for title and subtitle, they should be used instead of the original version.
These translations can be added in the Translations app.


TODO

  • Cypress tests
  • Update docs
  • Manual testing
  • task

Screenshots

Before:
Screenshot 2023-12-07 at 16 00 08

After:
Screenshot 2023-12-07 at 16 04 42

@dhis2-bot
Copy link
Contributor

dhis2-bot commented Dec 7, 2023

@dhis2-bot dhis2-bot temporarily deployed to netlify December 7, 2023 15:10 Inactive
Copy link

cypress bot commented Dec 7, 2023

1 flaky test on run #2691 ↗︎

0 388 1 0 Flakiness 1

Details:

Merge f33343d into 863b3b5...
Project: Data Visualizer App Commit: cb28bb70ae ℹ️
Status: Passed Duration: 06:10 💡
Started: Mar 6, 2024 2:50 PM Ended: Mar 6, 2024 2:57 PM
Flakiness  cypress/integration/dimensions/data.cy.js • 1 flaky test • e2e-chrome-parallel-2.40

View Output Video

Test Artifacts
Data dimension > displays Event data items correctly Test Replay Screenshots Video

Review all test suite changes for PR #2929 ↗︎

@dhis2-bot dhis2-bot temporarily deployed to netlify March 5, 2024 12:31 Inactive
@edoardo edoardo marked this pull request as ready for review March 5, 2024 13:12
@dhis2-bot dhis2-bot temporarily deployed to netlify March 5, 2024 17:37 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify March 6, 2024 14:46 Inactive
@edoardo edoardo merged commit 6f14116 into dev Mar 6, 2024
17 of 18 checks passed
@edoardo edoardo deleted the fix/title-translation-DHIS2-16216 branch March 6, 2024 15:00
dhis2-bot added a commit that referenced this pull request Mar 19, 2024
# [100.5.0](v100.4.0...v100.5.0) (2024-03-19)

### Bug Fixes

* **translations:** sync translations from transifex (dev) ([dbb683f](dbb683f))
* fetch displayTitle and displaySubtitle for translations DHIS2-16216 ([#2929](#2929)) ([d6f327a](d6f327a))
* **push-analytics:** add push analytics instructions ([#2985](#2985)) ([77c5bcd](77c5bcd))
* **translations:** sync translations from transifex (dev) ([476e8a8](476e8a8))
* **translations:** sync translations from transifex (dev) ([459c53c](459c53c))
* **translations:** sync translations from transifex (dev) ([2966187](2966187))
* add custom errors for analytics requests ([#2874](#2874)) ([cc3fcd1](cc3fcd1))

### Features

* add class-names for push-analytics ([#2966](#2966)) ([5872a5a](5872a5a))
* support Outlier table visualization type (DHIS2-13858) ([#2942](#2942)) ([e9c31a7](e9c31a7))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants